Skip to content

fix: Upload packaged assets when asset packaging is enabled#28

Merged
NerdsCorp merged 1 commit into
mainfrom
claude/fix-asset-packaging-upload-uTxBb
Dec 28, 2025
Merged

fix: Upload packaged assets when asset packaging is enabled#28
NerdsCorp merged 1 commit into
mainfrom
claude/fix-asset-packaging-upload-uTxBb

Conversation

@NerdsCorp

Copy link
Copy Markdown
Owner

When the "Package Update Assets" setting is enabled, the system now correctly uploads the packaged assets from resources/packs/ instead of the original unpackaged files.

Changes:

  • Modified BuildExclusionLists to accept packagingEnabled parameter
  • When packaging is enabled for editor uploads:
    • Include resources/packs directory in upload
    • Exclude source texture/sound/music files that were packed
  • When packaging is disabled for editor uploads:
    • Maintain original behavior (exclude packs directory)
  • Updated PerformUpload to pass packaging enabled status

This fixes the issue where packaged assets were created but not uploaded to the server.

When the "Package Update Assets" setting is enabled, the system now correctly uploads the packaged assets from resources/packs/ instead of the original unpackaged files.

Changes:
- Modified BuildExclusionLists to accept packagingEnabled parameter
- When packaging is enabled for editor uploads:
  - Include resources/packs directory in upload
  - Exclude source texture/sound/music files that were packed
- When packaging is disabled for editor uploads:
  - Maintain original behavior (exclude packs directory)
- Updated PerformUpload to pass packaging enabled status

This fixes the issue where packaged assets were created but not uploaded to the server.
@NerdsCorp NerdsCorp merged commit 42c07a0 into main Dec 28, 2025
@NerdsCorp NerdsCorp deleted the claude/fix-asset-packaging-upload-uTxBb branch December 28, 2025 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants